Skip to content

Add method for determining the size of the context #855

Merged
merged 2 commits into from
Aug 16, 2022
Merged

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Aug 15, 2022

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Look #844.

Closing issues

closes #844

@Mr-Geekman Mr-Geekman self-assigned this Aug 15, 2022
@github-actions
Copy link

🚀 Deployed on https://deploy-preview-855--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request August 15, 2022 10:30 Inactive
@codecov-commenter
Copy link

Codecov Report

Merging #855 (06e878d) into inference (7ef6231) will increase coverage by 0.11%.
The diff coverage is 93.82%.

@@              Coverage Diff              @@
##           inference     #855      +/-   ##
=============================================
+ Coverage      84.64%   84.75%   +0.11%     
=============================================
  Files            130      130              
  Lines           7450     7460      +10     
=============================================
+ Hits            6306     6323      +17     
+ Misses          1144     1137       -7     
Impacted Files Coverage Δ
etna/models/base.py 87.54% <84.61%> (-0.19%) ⬇️
etna/models/sarimax.py 94.81% <91.35%> (+1.16%) ⬆️
etna/models/deadline_ma.py 91.56% <95.00%> (+4.26%) ⬆️
etna/datasets/tsdataset.py 90.76% <100.00%> (+0.07%) ⬆️
etna/models/autoarima.py 100.00% <100.00%> (+9.58%) ⬆️
etna/models/catboost.py 100.00% <100.00%> (ø)
etna/models/holt_winters.py 98.86% <100.00%> (+0.04%) ⬆️
etna/models/nn/deepar.py 100.00% <100.00%> (ø)
etna/models/nn/tft.py 100.00% <100.00%> (ø)
etna/models/prophet.py 98.85% <100.00%> (+0.04%) ⬆️
... and 5 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Mr-Geekman Mr-Geekman changed the base branch from inference to master August 15, 2022 11:12
@Mr-Geekman Mr-Geekman changed the base branch from master to inference August 15, 2022 11:12
Copy link
Contributor

@martins0n martins0n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Mr-Geekman Mr-Geekman merged commit a39c8ef into inference Aug 16, 2022
@Mr-Geekman Mr-Geekman deleted the issue-844 branch August 16, 2022 09:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants